Feature: Enhance activity logging with context and subject relations - #55
Draft
lcharette wants to merge 31 commits into
Draft
Feature: Enhance activity logging with context and subject relations#55lcharette wants to merge 31 commits into
lcharette wants to merge 31 commits into
Conversation
…ations - Added new fields to the activity model: context_id, context_type, subject_id, subject_type, and metadata. - Added new ActivityRecorder service to handle activity logging with context and subject relations. - Refactored code to use ActivityRecorder instead of UserActivityLogger. - Updated UserActivityLoggerTest to mark it as deprecated. - Introduced GroupActivityTypes and RoleActivityTypes enums for structured activity types. - Added MorphableModelInterface to define polymorphic model contracts and updated existing models and interfaces to implement MorphableModelInterface.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## 6.1 #55 +/- ##
============================================
Coverage 100.00% 100.00%
- Complexity 2883 2968 +85
============================================
Files 559 578 +19
Lines 11989 12468 +479
Branches 671 689 +18
============================================
+ Hits 11989 12468 +479 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
…te tracking, and improve metadata for profile and group edits
… new values of attributes. - Modified ActivityInterface to reflect changes in user_id and added properties. - Adjusted use of 'subject' instead of 'context'. - Introduced ActivityDescription component for rendering activity descriptions and property changes in the UI. - Made user nullable in Activity Table - Etc.
…r conditional property tracking
- Updated the admin user interface to utilize dedicated forms for profile, group, password, role, status, and verification changes. - Removed group assignment from user creation. - Introduced UserGroupForm and UserGroupModal components for managing user groups. - Replaced UserActivateModal with UserStatusModal and UserVerificationModal for better clarity in user status management. - Updated UserForm to remove group selection and adjusted related components accordingly. - Refactored API calls in user management modals to use specific functions for user status, verification, and group updates. - Enhanced tests to cover new modal and form functionalities.
…nd update related components
Member
Author
|
@copilot resolve the merge conflicts in this pull request |
# Conflicts: # packages/sprinkle-admin/CHANGELOG.md # packages/sprinkle-admin/app/locale/en_US/messages.php # packages/sprinkle-admin/app/locale/fr_FR/messages.php # packages/sprinkle-core/app/locale/en_US/messages.php # packages/sprinkle-core/app/locale/fr_FR/messages.php Co-authored-by: lcharette <2566513+lcharette@users.noreply.github.com>
Contributor
Resolved the merge conflicts and preserved the activity translations in the new YAML locale files. Merge commit: |
…rove code clarity
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Implementation of #43
TODO :